Hi @agvandervelde , do I understand correctly that you list all your package's dependencies in additional_dependenies, just like for the roxygenise hook? Or do you override language: r? In any case, that needs to be documented better. I'll try to do amend your PR to that effect.

Yes, I list the dependencies in additional_dependenies. We're doing that already for styler so it's not an additional burden in our case. Thank you for looking into this!

For styler? I don’t get it. How would the styler hook need additional dependencies? You mean roxygen? This needs to load the package as per the docs.

For styler? I don’t get it. How would the styler hook need additional dependencies? You mean roxygen? This needs to load the package as per the docs.

Apologies, yes, roxygen.

Hi @agvandervelde, sorry this took a while... I pushed some (mildly related) doc changes to this branch too and I think this should be mergeable soon. Thanks for your work.

Thank you!

Rebased this PR onto current main.

Sorry did not get to merge this yet. Hope everything passes now.

@agvandervelde thanks again. If you don’t want to wait till the next tag bump, you can also pre-commit autoupdate --bleeding-edge or similar to use the lastest main commit.